Skip to content

fix: macos prevent circular mapping#90

Merged
thisislvca merged 2 commits intoTomBadash:masterfrom
farfromrefug:fix/macos_prevent_circular_mapping
Apr 21, 2026
Merged

fix: macos prevent circular mapping#90
thisislvca merged 2 commits intoTomBadash:masterfrom
farfromrefug:fix/macos_prevent_circular_mapping

Conversation

@farfromrefug
Copy link
Copy Markdown
Contributor

In my config i had:

  • middle -> mode_shift
  • mode_shift -> middle

This was creating a circular mapping and i could not get mode_shift -> middle to work.

Now we mark cgevents as "injected" and thus it wont try to map it again

@thisislvca
Copy link
Copy Markdown
Collaborator

Thanks for the contribution on this one. I pushed a tiny follow-up fix for the unrelated Linux KEY_VOLUMEUP typo that was breaking CI, so the PR should be in a better spot now.

@farfromrefug
Copy link
Copy Markdown
Contributor Author

@thisislvca thanks i dont know where that KEY_VOLUMEUP change comes from. i have the same in another PR

@thisislvca
Copy link
Copy Markdown
Collaborator

No worries, I traced it back and the KEY_VOLUME_UP rename was introduced in da75644 on this PR. Before that the file still used KEY_VOLUMEUP, so it looks like an accidental rename :)

@thisislvca thisislvca merged commit 19d8014 into TomBadash:master Apr 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants